Skip to content

fix(review): address follow-up IPC and boundary issues#33

Open
MerhiOPS wants to merge 5 commits intofix/security-review-findings-severity-batchesfrom
fix/review-followups-focused
Open

fix(review): address follow-up IPC and boundary issues#33
MerhiOPS wants to merge 5 commits intofix/security-review-findings-severity-batchesfrom
fix/review-followups-focused

Conversation

@MerhiOPS
Copy link
Contributor

Summary

  • reserve the internal plugin IPC namespace across voltkit and the runtime IPC module
  • split plugin storage persistence and plugin process transport helpers into focused modules
  • split volt-core security validation from CSP policy and apply rust formatting

Validation

  • pnpm --filter voltkit test
  • pnpm --filter voltkit lint
  • pnpm --filter voltkit typecheck
  • cargo fmt --all -- --check
  • cargo clippy -p volt-core -- -D warnings
  • cargo clippy -p volt-runner -- -D warnings
  • cargo test -p volt-core --test security_fs_integration -- --nocapture
  • cargo test -p volt-core --lib security:: -- --nocapture
  • cargo test -p volt-core
  • cargo test -p volt-runner plugin_manager::tests::storage -- --nocapture
  • cargo test -p volt-runner plugin_manager::tests::request_access -- --nocapture
  • cargo test -p volt-runner bounded_stderr_reader_caps_capture_size -- --nocapture
  • cargo test -p volt-runner drain_waiters_disconnects_pending_receivers -- --nocapture
  • cargo test -p volt-runner ipc_main_rejects_reserved -- --nocapture
  • cargo test -p volt-runner ipc_bridge::response::tests -- --nocapture
  • cargo test -p volt-runner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant